Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly refines the project's contribution guidelines by overhauling the GitHub issue and pull request templates. The changes aim to provide clearer, more structured pathways for users to report bugs, request features, ask questions, and submit code contributions, thereby improving the overall quality and efficiency of community interactions and streamlining the review process. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request significantly improves the contribution experience by introducing structured issue templates for bug reports, feature requests, and questions. It also updates the pull request template with a clearer checklist and correct repository links. The changes enhance the clarity and guidance for contributors, streamlining the process of reporting issues and submitting pull requests.
| --> | ||
|
|
||
| This PR [briefly explain what is does] | ||
| Related issue: https://github.com/proxysql/orchestrator/issues/ |
There was a problem hiding this comment.
To make it clearer for contributors, consider adding a placeholder for the issue number in the Related issue link. This would guide them on where to insert the relevant issue ID.
| Related issue: https://github.com/proxysql/orchestrator/issues/ | |
| Related issue: https://github.com/proxysql/orchestrator/issues/<ISSUE_NUMBER> |
There was a problem hiding this comment.
Pull request overview
Updates the repository’s GitHub contribution UX by replacing the single legacy issue template with three categorized templates and modernizing the pull request template to match the proxysql/orchestrator fork and expected contribution checks.
Changes:
- Replace
.github/ISSUE_TEMPLATE.mdwith separate Bug Report, Feature Request, and Question templates under.github/ISSUE_TEMPLATE/. - Update the PR template to reference
proxysql/orchestratorand add a contribution checklist (formatting, tests, CI, DCO, issue link).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/PULL_REQUEST_TEMPLATE.md | Rewords the intro comment, updates issue URL base, and adds a checklist; introduces a reference to CONTRIBUTING.md. |
| .github/ISSUE_TEMPLATE/bug_report.md | Adds a structured bug report template requesting env/config/topology/logs. |
| .github/ISSUE_TEMPLATE/feature_request.md | Adds a structured feature request template (problem/solution/alternatives/context). |
| .github/ISSUE_TEMPLATE/question.md | Adds a question template with a docs link and basic section header. |
| .github/ISSUE_TEMPLATE.md | Removes the old single-file issue template. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Please read CONTRIBUTING.md before submitting your PR. | ||
|
|
There was a problem hiding this comment.
The template asks contributors to read CONTRIBUTING.md, but that file doesn’t exist in the repository right now, so this will be a dead reference. Either add CONTRIBUTING.md in this PR, or change this line to link to an existing contribution guide (or remove it until the file lands).
| Please read CONTRIBUTING.md before submitting your PR. |
| --> | ||
|
|
||
| This PR [briefly explain what is does] | ||
| Related issue: https://github.com/proxysql/orchestrator/issues/ |
There was a problem hiding this comment.
Related issue: currently points to the repository’s issue list, not to an actual issue (and doesn’t provide a placeholder format). Update this to an issue reference placeholder that GitHub will auto-link (e.g., Related issue: #<number> / Closes #<number>), so PRs consistently connect to a specific tracking issue.
| Related issue: https://github.com/proxysql/orchestrator/issues/ | |
| Related issue: #<issue-number> |
| labels: question | ||
| --- | ||
|
|
||
| > **Note:** Please check the [orchestrator documentation](https://github.com/proxysql/orchestrator/tree/master/docs) before asking your question — it may already be answered there. |
There was a problem hiding this comment.
This documentation link hard-codes the master branch. If the default branch is main (or changes again later), this will break. Prefer a branch-agnostic relative link to the docs directory (or a URL that tracks the default branch).
| > **Note:** Please check the [orchestrator documentation](https://github.com/proxysql/orchestrator/tree/master/docs) before asking your question — it may already be answered there. | |
| > **Note:** Please check the [orchestrator documentation](../../docs) before asking your question — it may already be answered there. |
41af992 to
87a8517
Compare
Summary
.github/ISSUE_TEMPLATE.mdwith three structured templates: bug report, feature request, and question.github/PULL_REQUEST_TEMPLATE.mdto referenceproxysql/orchestrator, link toCONTRIBUTING.md, and include a proper checklist (gofmt, tests, CI, DCO sign-off, issue link)Closes #4
Test plan
ISSUE_TEMPLATE.mdis removed